Skip to content
This repository has been archived by the owner on Dec 26, 2022. It is now read-only.

fix(mam): Move definition to avoid SEGV fault #726

Merged
merged 1 commit into from
Jul 21, 2020

Conversation

howjmay
Copy link
Contributor

@howjmay howjmay commented Jul 21, 2020

Define mam_encrypt_key_t object in the middle of the function would
cause SEGV when the mam_encrypt_key_t object being freed.

And using ta_error_to_string to generate the string which is going to
be printed in logger.

For #706

Copy link
Member

@jserv jserv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Improve the subject of this commit. Make it meaningful.

@howjmay howjmay changed the title fix(mam): Fix errors in MAM implementation fix(mam): Move definition to avoid SEGV fault Jul 21, 2020
Define `mam_encrypt_key_t` object in the middle of the function would
cause SEGV when the `mam_encrypt_key_t` object being freed.

And using `ta_error_to_string` to generate the string which is going to
be printed in logger.

For DLTcollab#706
@howjmay howjmay added the C-bug Category - Something isn't working label Jul 21, 2020
@jserv jserv merged commit 3c5352c into DLTcollab:develop Jul 21, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
C-bug Category - Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants